sizerequest: Fix for_size adjustment

We need to pull the proper size from the size request cache and adjust
it accordingly.

box: Simplify some code

All out varibles in measure implementations must be non-null and -1 is
the default value for the baseline parameters.

box: Remove self assignment

box: Remove useless if statement

We already check right before this one whether child->pack != packing
and if so, we continue to the next iteration. So, no need to check again
whether the inverted condition child->pack == packing is true, because
it is.

label: Update layout width directly form allocation

For the one update_layout_width call in size_allocate, we can just use
the passed-in allocation width instead of a separate (relatively slow)
gtk_widget_get_width call.

widget: Clip redrawn area to widget clip

The clip can already be modified in size_allocate, so we never want a
widget to invalidate more than its actual clip region.

box: Remove some unneeded checks

We only call this function from the measure implementation and we should
never get NULL pointers there.

box: Remove an unnecessary local variable

box: Use GtkWidget's widget list in get_size

listbox: Don't assume that row child == focus widget

With widgets supporting child widgets and custom GtkListBoxRow
subclasses, this might not be true.

widget: Remove reparent special case

Since gtk_widget_reparent doesn't exist anymore, we can also remove this
special case.

container: Remove some unused includes

viewport: Remove some unused includes

gl area: Avoid trivialities

If we have zero pixels, no need to snapshot anything,
and produce GL errors while doing so.

gtk-demo: Add gears to the bowl

Just for fun.

Drop GtkGLArea::has-alpha

We now always behave as if has-alpha is TRUE.

Update all callers.

Convert gtkglarea to use snapshots

Just append a texture node.

Note that this is not 100% done yet. The GL area really
needs to keep a pool of textures, and only reuse them
once the GdkTexture object is gone.

gsk: Handle GL textures

These textures already have a GL texture in them,
no need to create a new one.

gdk: Add a gl texture implementation

This will be used to pass a GL textures from the application
(or rather, GtkGLArea) down to the GSK GL renderer.

Drop removed api from the docs

This should have been part of the previous commit.

gdk: Drop some unused cursor apis

The query function for cursor sizes and capabilities
are not very interesting. At least, they are not used
in GTK+, and all backends but X11 just hardcode
made-up values anyway. So, lets drop them.

inspector: Stop using gdk_display_get_default_cursor_size

If the cursor theme size setting has no value, just hardcode 32.
Also, allow ridiculously large cursors, just for fun.

tooltip: Stop using gdk_display_get_default_cursor_size

It makes more sense to use the actual cursor theme size here.

testgtk: Stop using gdk_display_get_maximal_cursor_size

This function is going away. Just make a reasonable assumption
that cursor sizes up to 128 are supported.

gtk/queryimmodules.c: Make the output deterministic.

Whilst working on the Reproducible Builds effort [0], we noticed that
queryimmodules generates non-reproducible output as it iterates over the
filesystem without sorting.

Patch attached.

 [0] https://reproducible-builds.org/

Signed-off-by: Chris Lamb <lamby@debian.org>
https://bugzilla.gnome.org/show_bug.cgi?id=786528

tree model sort: Fix set_sort_column

We were failing to change the sort order for the
default sort column in some cases. Fix that, and
add a testcase for this issue.

https://bugzilla.gnome.org/show_bug.cgi?id=792459

Add a testcase for the previous fix

tree model sort: Fix initial default sort func

gtk_tree_sortable_has_default_sort_func should return
FALSE initially.

Fix the icontheme test

We rely on log messages here. Since logging is per-display
now, we need to set a display on our custom icontheme object
to get the expected log messages.

mountoperation: Fix a crash

We only create the gesture when the dialog is needed,
so don't free it unconditionally.

Drop ::button-press/release event

We are no longer using these, and there are several gestures
that can be used instead. If you need to catch raw button events,
use the ::event signal.

popover: Stop using ::button-press/release-event

Just use a generic ::event handler.

menushell: Stop using ::button-press/release-event

Merge the handlers into the existing handler for ::event.

menu: Remove stray references to 'button_press'

No code change, but this string was still used in
docs and comments.

places sidebar: Stop using ::button-press/release-event

We'll use a generic ::event signal instead.

gtk-demo: Stop using ::button-release-event

tests: Stop using ::button-press/release-event

The generic ::event signal works fine here.

testsuite: Stop using ::button-press-event

We can use the generic ::event signal instead.

tests: Stop using ::button-press-event

A gesture will do here.

Drop ::motion-notify-event

We no longer emit this signal. You can use various gestures
and event controllers instead. If you need to catch raw
motion events, use the generic ::event signal.

places sidebar: Stop using ::motion-notify-event

We can use the generic ::event signal here.

docs: Stop referring to specific event signals

These are going away.

gtk-demo: Stop using ::motion-notify-event in the hypertext demo

We already have a generic ::event handler, just use it for this.

tests: Stop using ::motion-notify-event

We can use the generic ::event signal here.

treeview: Stop using ::button-press-event

We can just use a gesture here.

recent chooser: Stop using ::button-press-event

We can just use a multipress gesture here.

mount operation: Stop using ::button-press-event

A multipress gesture does just fine here.

testinput: Port to a drag gesture

This replaces both button-press-event and motion-notify-event
handlers.

entry completion: Drop another ::button-press-event handler

This one will be replaced by a generic ::event handler.

docs: Stop referring to legacy event signals

These are going away.

entry completion: Stop using ::button-press-event

Just use the existing ::row-activated signal on the treeviews.

gtk-demo: Stop using ::button-press-event

Use gestures instead.

testgtk: Stop using ::button-press-event

This one example can just as well use a gesture.

places view: Stop using ::button-press-event

Instead, use a multi-press gesture in the row widget,
and emit ::popup-menu from there.

file chooser: Use a multipress gesture

This replaces the use of ::button-press-event. There's two
issues with this commit:
1) We don't have a good way to do the equivalent of
   gdk_event_triggers_context_menu with gestures
2) We have to defer to and idle to avoid ordering
   issues with the treeviews own gestures

Revert "file chooser: Allow activating without double-click"

This reverts commit fb0a13b7f070a14312dafa1e4df6ba03cf33be01.

It is getting in the way of gesture conversion, and didn't
really make anybody happy anyway.

app chooser: Use a gesture

Instead of connecting to ::button-press-event, use
a multipress gesture here.

Remove a leftover icon-size property

This made the 'style classes' demo crash.

Drop ::focus-in/out-event

These signals are no longer used in GTK+.
If you need to catch focus events, use the ::event signal.

treeview: Stop using ::focus-out-event

We can just use the generic ::event signal.

file chooser entry: Stop using ::focus-out-event

Just use the generic ::event signal.

spin button: Stop using ::focus-out-event

Just use the generic ::event signal.

calendar: Stop using ::focus-out-event

Use the generic ::event signal.

color editor: stop using ::focus-out-event

Again, the has-focus property is sufficient.

text cell renderer: Stop using ::focus-out-event

The has-focus property works fine here.

spin cell renderer: Stop using ::focus-out-event

The has-focus property works fine here.

combo cell renderer: Stop using ::focus-out-event

The has-focus property works fine here.

tests: Stop using ::focus-in/out-event

The has-focus property works just as well here.

popover: Stop using ::focus-in/out-event

We can use the is-active property of the window instead.

application: Stop using ::focus-in-event

We are really interested in the active window, so use
that property directly.

window: Stop using ::focus-in/out-event

We can just use the generic ::event.

text view: Stop using ::focus-in/out-event

We can just use the generic ::event here.

entry completion: Stop using ::focus-out-event

We can just use ::event.

entry: Stop using ::focus-in/out-event

We can just use the generic ::event signal.

a11y: Stop using ::focus-in/out-event

We can just use the generic ::event signal.

Drop gtk_widget_hide_on_delete

This function is misnamed - it is only ever relevant for
windows. And with the ::hide-on-delete property , it is
no longer necessary to use the signal for this simple case.

Drop ::event-after

This signal is not used in GTK+, and we are reducing the
amount of widget signals devoted to events in favor of
event controllers.

Drop ::grab-broken-event

This signal is no longer used in GTK+. Grab broken events
can be handled with the generic ::event signal.

button: Stop using ::grab-broken-event

The event-specific signals are going away. Just use ::event here.

menushell: Stop using ::grab-broken

The event-specific signals are going away. Just use ::event here.

notebook: some cleanup

notebook: Use a motion event controller

This lets us avoid the ::motion-notify-event handler.

textview: Use a motion event controller

This lets us replace the ::motion-notify-event handler.

Drop ::enter/leave-notify-event

These signals are no longer used in GTK+ and have a (not quite
perfect yet) replacement with GtkEventControllerMotion.

If you need to catch the raw events, the generic ::event
signal still works.

menu: Use a motion event controller

This lets us replace the ::enter/leave-notify-event handlers.
Not that there is a FIXME here - we currently can't get the
crossing mode from the event controller, so we have to fall
back to gtk_get_current_event().

menuitem: Use a motion event controller

This lets us replace the ::enter/leave-notify-event handlers.
Not that there is a FIXME here - we currently can't get the
crossing mode from the event controller, so we have to fall
back to gtk_get_current_event().

iconview: Use a motion event controller

This can replace ::motion-notify-event and ::leave-notify-event.

scrolled window: Use a motion event controller

This lets use replace one of the last uses of ::leave-notify-event.

entry completion: Drop initial selection avoidance

This is a minor thing, and it is hard to recreate
with event controllers, so drop it for now.

treeview: Use a motion event controller

This replaces handlers for ::enter/leave/motion-notify-event.

Add x/y to GtkEventControllerMotion::enter

We have this information available in enter events,
and having it in the controller signal as well makes
porting easier.

Update existing users.

Drop ::configure-event

This signal is unused in GTK+ and configure events are handled
internally by GtkWindow.

If you need to catch configure events, the generic ::event
signal still works.

window: Stop using ::configure-event

We already have a generic ::event handler that we
can just press into service for this.

testgtk: Stop using ::configure-event

xim: Stop using ::configure-event

The generic ::event will do.

tests: Stop using ::configure-event

The generic ::event will do.

Drop ::touch-event

This signal is not used in GTK+, and has a suitable
replacement with GtkGesture and its subclasses.

If you need to catch the raw touch events, the generic ::event
signal still works, too.

Drop ::scroll-event

This signal is not used in GTK+, and has a suitable
replacement with GtkEventControllerScroll.

If you need to catch these events, the generic ::event
signal still works, too.

Drop ::proximity-in-event and ::proximity-out-event

These are unused in GTK+, and are not very common.
If you need to catch these events, the generic ::event
signal still works.

Drop ::map-event and ::unmap-event

These are unused in GTK+, and are not very common.
If you need to catch these events, the generic ::event
signal still works.

window: Stop using ::map-event

Use ::event here, since we want to drop the event-specific
signals, and we already have a handler for the generic signal.